Skip to content

init markdown-edit-block#66

Merged
Wattenberger merged 9 commits intomainfrom
aw/markdown-edit-block
Aug 9, 2022
Merged

init markdown-edit-block#66
Wattenberger merged 9 commits intomainfrom
aw/markdown-edit-block

Conversation

@Wattenberger
Copy link
Copy Markdown
Contributor

@Wattenberger Wattenberger commented Aug 2, 2022

image

This PR also includes a new Sandbox Block, to replace the CodeSandbox components in the old Markdown preview Block.

Todo later:

  • add support for custom content for a BlockComponent
  • syntax highlighting for code snippets

@Wattenberger Wattenberger marked this pull request as draft August 2, 2022 23:19
".cm-cursor, .cm-dropCursor": { borderLeftColor: colors.cursor },
"&.cm-focused .cm-selectionBackground, .cm-selectionBackground, .cm-content ::selection":
{ backgroundColor: colors.bg },
{ backgroundColor: colors.selectionBg },
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

something I noticed poking around here - lil fix

@Wattenberger Wattenberger force-pushed the aw/markdown-edit-block branch from 7a1f065 to 07e590f Compare August 5, 2022 18:27
@Wattenberger Wattenberger marked this pull request as ready for review August 8, 2022 20:12
Copy link
Copy Markdown
Contributor

@mattrothenberg mattrothenberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking very cool. I wasn't able to dive too deeply into the CodeMirror bits, but it's working on my end

Comment thread blocks/file-blocks/markdown-edit/copy-widget.tsx Outdated
Comment thread blocks/file-blocks/sandbox/index.tsx
@Wattenberger Wattenberger merged commit eadd59e into main Aug 9, 2022
@Wattenberger Wattenberger deleted the aw/markdown-edit-block branch August 9, 2022 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants